home *** CD-ROM | disk | FTP | other *** search
/ Usenet 1993 July / InfoMagic USENET CD-ROM July 1993.ISO / sources / x / volume7 / xlock / patch4 next >
Encoding:
Internet Message Format  |  1990-06-13  |  24.8 KB
  1. Path: uunet!tut.cis.ohio-state.edu!zaphod.mps.ohio-state.edu!brutus.cs.uiuc.edu!apple!sun-barr!newstop!sun!cellar.bae.bellcore.com)!aw@garage.bae.bellcore.com
  2. From: aw@garage.bae.bellcore.com (aw@cellar.bae.bellcore.com)
  3. Newsgroups: comp.sources.x
  4. Subject: v07i096: xlock -- lockscreen for X displays, Patch4, Part01/01
  5. Message-ID: <137221@sun.Eng.Sun.COM>
  6. Date: 13 Jun 90 22:54:06 GMT
  7. Sender: news@sun.Eng.Sun.COM
  8. Lines: 801
  9. Approved: argv@sun.com
  10.  
  11. Submitted-by: Andrew Wason (aw@cellar.bae.bellcore.com) <aw@garage.bae.bellcore.com>
  12. Posting-number: Volume 7, Issue 96
  13. Archive-name: xlock/patch4
  14.  
  15. [ Moderator's note: There is no Patch-To: line because the source has
  16.     passed through too many hands and revisions since it's original posting
  17.     in Volume 5.  So, let's assume the submitter of this patch is correct
  18.     and call the current version patch level 4.  that is, you should apply
  19.     this patch to version 1.3 of xlock...  Discrepincies should be taken
  20.     up with Andrew Wason, not me.  However, keep me appraised of any
  21.     conclusions for future postings. --dan ]
  22.  
  23. This is an unofficial patch to the contrib xlock program
  24. (patchlevel 1.3 - available from expo.lcs.mit.edu).
  25. I have tried to contact the author, but never received any response.
  26. If this isn't the correct place to send these patches,
  27. please let me know what is.
  28.  
  29. This patch fixes colormap problems xlock has when run under
  30. ICCCM compliant window managers (e.g. R4 twm).
  31. See the file README.patch for more information.
  32.  
  33. The shar file below contains README.patch and xlock.patch.
  34. Typing 'patch < xlock.patch' in the source code directory for
  35. xlock 1.3 will apply the patch.
  36.  
  37. --------------------------------------------------------------------------------
  38. Andrew Wason                                        Bell Communications Research
  39. aw@cellar.bae.bellcore.com                          Piscataway, NJ
  40. bellcore!cellar!aw
  41. --------------------------------------------------------------------------------
  42.  
  43.  
  44. #--------------------------- Cut Here --------------------------
  45. #!/bin/sh
  46. #
  47. # Run this file through sh
  48. #
  49. echo README.patch
  50. sed -e 's/^Z//' > README.patch << 'EOF of README.patch'
  51. Patching:
  52.  
  53.   This is an unofficial patch for version 1.3 of xlock.
  54.   It should only be applied if the file patchlevel.h contains the number 1.3.
  55.   This patch increases the patchlevel to 1.4.
  56.   I did not include a "Prereq:" line in the patch, because patch
  57.   core dumps when I do, so it's up to you to make sure you are patching
  58.   version 1.3.
  59.  
  60. Colormap changes:
  61.  
  62.      Xlock builds it's own colormap which contains colors which make the
  63.   qix, hop and life modes look appealing (e.g. the stick color in qix moves
  64.   through different hues as it moves around the screen).  Xlock version 1.3
  65.   used XInstallColormap()/XUninstallColormap() to forcibly install/uninstall
  66.   this colormap. This violates ICCCM standards and does not work with
  67.   ICCCM compliant window managers (e.g. R4 twm). An ICCCM compliant window
  68.   manager will print an error message and refuse to install the colormap.
  69.   This makes xlock's colors look like garbage.
  70.  
  71.      An ICCCM compliant client should use XSetWindowColormap() and let the
  72.   window manager install the colormap for it.  This will not work for xlock
  73.   because the xlock window is override_redirect and so the window manager
  74.   doesn't know it exists and never installs the colormap.
  75.  
  76.      To solve this problem, this unofficial version of xlock uses XAllocColor()
  77.   to attempt to allocate the colors it needs out of the default colormap.
  78.   How many colors xlock is able to allocate depends on what other clients
  79.   are running. If the root window contains a colorful image, for example,
  80.   then xlock will not get as many colors. Use the -v option to see how
  81.   many colors xlock is able to allocate. In most cases, xlock can allocate
  82.   enough colors to look decent.
  83.  
  84.       There is a new -install option to xlock which makes it behave the
  85.   way the 1.3 version did (i.e. use XInstallColormap()/XUninstallColormap()).
  86.   This is useful for non-ICCCM compliant window managers.
  87.  
  88.  
  89. Popup windows:
  90.  
  91.      This patch also fixes a problem xlock has with popup windows.
  92.   Xlock does not allow any new clients to connect to the locked display,
  93.   but if a client is already connected but unmapped, and this client maps
  94.   itself after the screen has been locked, then this client's window will
  95.   pop up on top of the xlock window and potentially burn-in the screen.
  96.   An example of this is the xconsole program which is unmapped until it gets
  97.   a console message. It then maps itself on top of the xlock window.
  98.  
  99.      This patched version of xlock raises the xlock window every 'timeout'
  100.   seconds. So any windows which have popped up will be covered again.
  101.  
  102.  
  103. Andrew Wason
  104. Bellcore
  105. aw@cellar.bae.bellcore.com
  106. EOF of README.patch
  107. echo "      55     418    2626 README.patch"
  108. wc README.patch
  109.  
  110. echo xlock.patch
  111. sed -e 's/^Z//' > xlock.patch << 'EOF of xlock.patch'
  112. Index: patchlevel.h
  113. diff -c xlock.orig/patchlevel.h xlock.new/patchlevel.h
  114. *** xlock.orig/patchlevel.h    Thu Jun  7 09:49:42 1990
  115. --- xlock.new/patchlevel.h    Wed Jun 13 16:03:40 1990
  116. ***************
  117. *** 1 ****
  118. ! 1.3
  119. --- 1 ----
  120. ! 1.4
  121. diff -c xlock.orig/HSBmap.c xlock.new/HSBmap.c
  122. *** xlock.orig/HSBmap.c    Thu Jun  7 09:49:45 1990
  123. --- xlock.new/HSBmap.c    Fri May 18 19:26:35 1990
  124. ***************
  125. *** 110,118 ****
  126.       r = red;
  127.       g = green;
  128.       b = blue;
  129. !     dh = (h2 - h1) / 255.0;
  130. !     ds = (s2 - s1) / 255.0;
  131. !     db = (b2 - b1) / 255.0;
  132.       for (i = start; i <= end; i++) {
  133.       hsb2rgb(h1, s1, b1, r++, g++, b++);
  134.       h1 += dh;
  135. --- 110,118 ----
  136.       r = red;
  137.       g = green;
  138.       b = blue;
  139. !     dh = (h2 - h1) / (double) (end - start);
  140. !     ds = (s2 - s1) / (double) (end - start);
  141. !     db = (b2 - b1) / (double) (end - start);
  142.       for (i = start; i <= end; i++) {
  143.       hsb2rgb(h1, s1, b1, r++, g++, b++);
  144.       h1 += dh;
  145. diff -c xlock.orig/README xlock.new/README
  146. *** xlock.orig/README    Thu Jun  7 09:49:41 1990
  147. --- xlock.new/README    Wed Jun  6 17:05:36 1990
  148. ***************
  149. *** 1,3 ****
  150. --- 1,13 ----
  151. + This is an unofficially patched version of xlock version 1.3
  152. + 
  153. + CHANGES:
  154. +    o use the default colormap instead of forcibly installing a colormap
  155. +    o forcibly install the colormap if the -install option is used
  156. +    o raise the xlock window every 'timeout' seconds (to cover any windows
  157. +      which have popped up)
  158. + 
  159. + 
  160. + ----------------- version 1.3 README ------------------------------------
  161.   
  162.   This is the latest version of xlock (sccs version 22.7) patchlevel = 1.3
  163.   
  164. diff -c xlock.orig/XCrDynCmap.c xlock.new/XCrDynCmap.c
  165. *** xlock.orig/XCrDynCmap.c    Thu Jun  7 09:49:45 1990
  166. --- xlock.new/XCrDynCmap.c    Fri May 18 19:01:46 1990
  167. ***************
  168. *** 29,85 ****
  169.   #include <X11/Xutil.h>
  170.   
  171.   Status
  172. ! XCreateDynamicColormap(dsp, screen, cmap, visual, colors,
  173. !                count, red, green, blue)
  174. !     Display    *dsp;
  175. !     int        screen;
  176. !     Colormap   *cmap;
  177. !     Visual    **visual;
  178. !     XColor     *colors;
  179. !     int        count;
  180. !     u_char     *red,
  181. !            *green,
  182. !            *blue;
  183.   {
  184.       XVisualInfo vinfo;
  185. -     int        pixels[256];
  186.       int        i,
  187. -         ncolors,
  188.           planes;
  189. !     unsigned long pmasks;
  190. !     Status    allocReturn;
  191.   
  192.       planes = DisplayPlanes(dsp, screen);
  193.       if (XMatchVisualInfo(dsp, screen, planes, PseudoColor, &vinfo)) {
  194.   
  195.       *visual = vinfo.visual;
  196. !     *cmap = XCreateColormap(dsp, RootWindow(dsp, screen),
  197. !                 *visual, AllocNone);
  198. !     ncolors = vinfo.colormap_size;
  199.   
  200. !     if (count > ncolors)
  201. !         return BadValue;
  202.   
  203. !     allocReturn = XAllocColorCells(dsp, *cmap,
  204. !                        False, &pmasks, 0,
  205. !                        pixels, count);
  206.   
  207. ! /*    This should return Success, but it doesn't... Xlib bug?
  208. !  *    (I'll ignore the return value for now...)
  209. !  */
  210. ! #ifdef NOTDEF
  211. !     if (allocReturn != Success)
  212. !         return allocReturn;
  213. ! #endif                /* NOTDEF */
  214.   
  215. !     for (i = 0; i < count; i++) {
  216. !         colors[i].pixel = pixels[i];
  217. !         colors[i].red = *red++ << 8;
  218. !         colors[i].green = *green++ << 8;
  219. !         colors[i].blue = *blue++ << 8;
  220. !         colors[i].flags = DoRed | DoGreen | DoBlue;
  221.       }
  222. !     XStoreColors(dsp, *cmap, colors, count);
  223.       return Success;
  224.       } else
  225.       return BadMatch;
  226. --- 29,86 ----
  227.   #include <X11/Xutil.h>
  228.   
  229.   Status
  230. ! XCreateDynamicColormap(dsp, screen, cmap, install, visual, red, green, blue,
  231. !         pixels, npixels)
  232. !     Display        *dsp;
  233. !     int            screen;
  234. !     Colormap        *cmap;
  235. !     int            install;
  236. !     Visual        **visual;
  237. !     u_char        *red,
  238. !             *green,
  239. !             *blue;
  240. !     unsigned long    pixels[256];    /* returned array of useable pixels 
  241. !                        in default colormap */
  242. !     int            *npixels;
  243.   {
  244.       XVisualInfo vinfo;
  245.       int        i,
  246.           planes;
  247. !     XColor    xcolors[256];
  248. !     char    flags;
  249.   
  250.       planes = DisplayPlanes(dsp, screen);
  251.       if (XMatchVisualInfo(dsp, screen, planes, PseudoColor, &vinfo)) {
  252.   
  253.       *visual = vinfo.visual;
  254. !     if (install)
  255. !         *cmap = XCreateColormap(dsp, RootWindow(dsp, screen),
  256. !                 *visual, AllocAll);
  257. !     else
  258. !         *cmap = DefaultColormap(dsp, screen);
  259.   
  260. !     *npixels = 0;
  261. !     flags = DoRed | DoGreen | DoBlue;
  262. !     for (i = 0; i < DisplayCells(dsp, screen); i++) {
  263. !         xcolors[i].red = *red++ << 8;
  264. !         xcolors[i].green = *green++ << 8;
  265. !         xcolors[i].blue = *blue++ << 8;
  266. !         xcolors[i].flags = flags;
  267.   
  268. !         if (install)
  269. !         xcolors[i].pixel = i;
  270.   
  271. !         /* if we successfuly alloc a color, save the pixel,
  272. !            if we fail then we are out of colormap space */
  273. !         else
  274. !         if (!XAllocColor(dsp, *cmap, &xcolors[i]))
  275. !             break;
  276.   
  277. !         pixels[i] = xcolors[i].pixel;
  278. !         (*npixels)++;
  279.       }
  280. !     if (install)
  281. !         XStoreColors(dsp, *cmap, xcolors, DisplayCells(dsp, screen));
  282.       return Success;
  283.       } else
  284.       return BadMatch;
  285. diff -c xlock.orig/XCrHsbCmap.c xlock.new/XCrHsbCmap.c
  286. *** xlock.orig/XCrHsbCmap.c    Thu Jun  7 09:49:45 1990
  287. --- xlock.new/XCrHsbCmap.c    Fri May 18 19:26:10 1990
  288. ***************
  289. *** 27,47 ****
  290.   #include <X11/Xos.h>
  291.   #include <X11/Xlib.h>
  292.   
  293. ! extern void HSBmap();
  294.   
  295.   Status
  296. ! XCreateHSBColormap(dsp, screen, cmap, count, h1, s1, b1, h2, s2, b2, bw)
  297. !     Display    *dsp;
  298. !     int         screen;
  299. !     Colormap   *cmap;        /* colormap return value */
  300. !     int         count;        /* number of entrys to use */
  301. !     double      h1,        /* starting hue */
  302. !                 s1,        /* starting saturation */
  303. !                 b1,        /* starting brightness */
  304. !                 h2,        /* ending hue */
  305. !                 s2,        /* ending saturation */
  306. !                 b2;        /* ending brightness */
  307. !     int         bw;        /* Boolean: True = save black and white */
  308.   {
  309.       u_char      red[256];
  310.       u_char      green[256];
  311. --- 27,50 ----
  312.   #include <X11/Xos.h>
  313.   #include <X11/Xlib.h>
  314.   
  315. ! extern void HSBramp();
  316.   
  317.   Status
  318. ! XCreateHSBColormap(dsp, screen, cmap, install, h1, s1, b1, h2, s2, b2, bw,
  319. !             pixels, npixels)
  320. !     Display      *dsp;
  321. !     int           screen;
  322. !     Colormap     *cmap;        /* if install, colormap return value */
  323. !     int           install;    /* true if we should install a colormap */
  324. !     double        h1,        /* starting hue */
  325. !                   s1,        /* starting saturation */
  326. !                   b1,        /* starting brightness */
  327. !                   h2,        /* ending hue */
  328. !                   s2,        /* ending saturation */
  329. !                   b2;        /* ending brightness */
  330. !     int           bw;        /* Boolean: True = save black and white */
  331. !     unsigned long pixels[256];    /* return array of useable pixels */
  332. !     int          *npixels;
  333.   {
  334.       u_char      red[256];
  335.       u_char      green[256];
  336. ***************
  337. *** 49,61 ****
  338.       unsigned long pixel;
  339.       Status      status;
  340.       Visual     *visual;
  341. -     XColor      xcolors[256];
  342.   
  343. !     if (count > 256)
  344. !     return BadValue;
  345.   
  346. -     HSBramp(h1, s1, b1, h2, s2, b2, 0, count - 1, red, green, blue);
  347. - 
  348.       if (bw) {
  349.       pixel = WhitePixel(dsp, screen);
  350.       red[pixel] = green[pixel] = blue[pixel] = 0xff;
  351. --- 52,61 ----
  352.       unsigned long pixel;
  353.       Status      status;
  354.       Visual     *visual;
  355.   
  356. !     /* create 3 256 element arrays of rgb values */
  357. !     HSBramp(h1, s1, b1, h2, s2, b2, 0, DisplayCells(dsp, screen) - 1, red, green, blue);
  358.   
  359.       if (bw) {
  360.       pixel = WhitePixel(dsp, screen);
  361.       red[pixel] = green[pixel] = blue[pixel] = 0xff;
  362. ***************
  363. *** 63,70 ****
  364.       pixel = BlackPixel(dsp, screen);
  365.       red[pixel] = green[pixel] = blue[pixel] = 0;
  366.       }
  367. !     status = XCreateDynamicColormap(dsp, screen, cmap, &visual, xcolors,
  368. !                     count, red, green, blue);
  369.   
  370.       return status;
  371.   }
  372. --- 63,70 ----
  373.       pixel = BlackPixel(dsp, screen);
  374.       red[pixel] = green[pixel] = blue[pixel] = 0;
  375.       }
  376. !     status = XCreateDynamicColormap(dsp, screen, cmap, install, &visual,
  377. !                     red, green, blue, pixels, npixels);
  378.   
  379.       return status;
  380.   }
  381. diff -c xlock.orig/hopalong.c xlock.new/hopalong.c
  382. *** xlock.orig/hopalong.c    Thu Jun  7 09:49:43 1990
  383. --- xlock.new/hopalong.c    Fri May 18 19:01:46 1990
  384. ***************
  385. *** 29,34 ****
  386. --- 29,35 ----
  387.    *               Mountain View, CA  94043
  388.    *
  389.    * Revision History:
  390. +  * 09-May-90: Use Pixels[] array to index into default colormap.
  391.    * 15-Dec-89: Fix for proper skipping of {White,Black}Pixel() in colors.
  392.    * 08-Oct-89: Fixed long standing typo bug in RandomInitHop();
  393.    *          Fixed bug in memory allocation in inithop();
  394. ***************
  395. *** 44,49 ****
  396. --- 45,53 ----
  397.   #include <X11/Xlib.h>
  398.   #include <X11/Xutil.h>
  399.   
  400. + extern unsigned long Pixels[];
  401. + extern int NPixels;
  402. + 
  403.   static int  centerx,
  404.           centery;        /* center of the screen */
  405.   static double a,
  406. ***************
  407. *** 152,162 ****
  408.       register XPoint *xp = pointBuffer;
  409.   
  410.       if (color) {
  411. !     XSetForeground(Dsp, Gc, pix++);
  412. !     while (pix >= DisplayCells(Dsp, DefaultScreen(Dsp)) ||
  413.           pix == WhitePixel(Dsp, DefaultScreen(Dsp)) ||
  414.           pix == BlackPixel(Dsp, DefaultScreen(Dsp))) {
  415. !         if (pix >= DisplayCells(Dsp, DefaultScreen(Dsp)))
  416.           pix = 0;
  417.           else
  418.           pix++;
  419. --- 156,166 ----
  420.       register XPoint *xp = pointBuffer;
  421.   
  422.       if (color) {
  423. !     XSetForeground(Dsp, Gc, Pixels[pix++]);
  424. !     while (pix >= NPixels ||
  425.           pix == WhitePixel(Dsp, DefaultScreen(Dsp)) ||
  426.           pix == BlackPixel(Dsp, DefaultScreen(Dsp))) {
  427. !         if (pix >= NPixels)
  428.           pix = 0;
  429.           else
  430.           pix++;
  431. diff -c xlock.orig/life.c xlock.new/life.c
  432. *** xlock.orig/life.c    Thu Jun  7 09:49:44 1990
  433. --- xlock.new/life.c    Fri May 18 19:01:46 1990
  434. ***************
  435. *** 30,35 ****
  436. --- 30,36 ----
  437.    *               Mountain View, CA  94043
  438.    *
  439.    * Revision History:
  440. +  * 09-May-90: Use Pixels[] array to index into default colormap.
  441.    * 07-Feb-90: remove bogus semi-colon after #include line.
  442.    * 15-Dec-89: Fix for proper skipping of {White,Black}Pixel() in colors.
  443.    * 08-Oct-89: Moved seconds() to an extern.
  444. ***************
  445. *** 41,46 ****
  446. --- 42,50 ----
  447.   #include <X11/Xutil.h>
  448.   #include "lifeicon.bit"
  449.   
  450. + extern unsigned long Pixels[];
  451. + extern int NPixels;
  452. + 
  453.   static XImage logo = {
  454.       0, 0,            /* width, height */
  455.       0, XYBitmap, 0,        /* xoffset, format, data */
  456. ***************
  457. *** 323,337 ****
  458.       register unsigned char *ageptr = agebuf + (loc - buffer);
  459.       register unsigned char age = *ageptr;
  460.   
  461. !     while (age >= DisplayCells(Dsp, DefaultScreen(Dsp)) ||
  462.           age == WhitePixel(Dsp, DefaultScreen(Dsp)) ||
  463.           age == BlackPixel(Dsp, DefaultScreen(Dsp))) {
  464. !         if (age >= DisplayCells(Dsp, DefaultScreen(Dsp)))
  465.           age = 0;
  466.           else
  467.           age++;
  468.       }
  469. !     XSetForeground(Dsp, Gc, age);
  470.       *ageptr = age + 1;
  471.       }
  472.       if (pixels)
  473. --- 327,341 ----
  474.       register unsigned char *ageptr = agebuf + (loc - buffer);
  475.       register unsigned char age = *ageptr;
  476.   
  477. !     while (age >= NPixels ||
  478.           age == WhitePixel(Dsp, DefaultScreen(Dsp)) ||
  479.           age == BlackPixel(Dsp, DefaultScreen(Dsp))) {
  480. !         if (age >= NPixels)
  481.           age = 0;
  482.           else
  483.           age++;
  484.       }
  485. !     XSetForeground(Dsp, Gc, Pixels[age]);
  486.       *ageptr = age + 1;
  487.       }
  488.       if (pixels)
  489. diff -c xlock.orig/qix.c xlock.new/qix.c
  490. *** xlock.orig/qix.c    Thu Jun  7 09:49:44 1990
  491. --- xlock.new/qix.c    Fri May 18 19:01:46 1990
  492. ***************
  493. *** 29,34 ****
  494. --- 29,35 ----
  495.    *               Mountain View, CA  94043
  496.    *
  497.    * Revision History:
  498. +  * 09-May-90: Use Pixels[] array to index into default colormap.
  499.    * 15-Dec-89: Fix for proper skipping of {White,Black}Pixel() in colors.
  500.    * 08-Oct-89: Fixed bug in memory allocation in initqix().
  501.    *          Moved seconds() to an extern.
  502. ***************
  503. *** 41,46 ****
  504. --- 42,50 ----
  505.   #include <X11/Xlib.h>
  506.   #include <X11/Xutil.h>
  507.   
  508. + extern unsigned long Pixels[];
  509. + extern int NPixels;
  510. + 
  511.   static Display *Dsp;
  512.   static Window Win;
  513.   static GC   Gc,
  514. ***************
  515. *** 156,166 ****
  516.       check_bounds_x(x2, &dx2);
  517.       check_bounds_y(y2, &dy2);
  518.       if (color) {
  519. !         XSetForeground(Dsp, Gc, pix++);
  520. !         while (pix >= DisplayCells(Dsp, DefaultScreen(Dsp)) ||
  521.               pix == WhitePixel(Dsp, DefaultScreen(Dsp)) ||
  522.               pix == BlackPixel(Dsp, DefaultScreen(Dsp))) {
  523. !         if (pix >= DisplayCells(Dsp, DefaultScreen(Dsp)))
  524.               pix = 0;
  525.           else
  526.               pix++;
  527. --- 160,170 ----
  528.       check_bounds_x(x2, &dx2);
  529.       check_bounds_y(y2, &dy2);
  530.       if (color) {
  531. !         XSetForeground(Dsp, Gc, Pixels[pix++]);
  532. !         while (pix >= NPixels ||
  533.               pix == WhitePixel(Dsp, DefaultScreen(Dsp)) ||
  534.               pix == BlackPixel(Dsp, DefaultScreen(Dsp))) {
  535. !         if (pix >= NPixels)
  536.               pix = 0;
  537.           else
  538.               pix++;
  539. diff -c xlock.orig/xlock.c xlock.new/xlock.c
  540. *** xlock.orig/xlock.c    Thu Jun  7 09:49:43 1990
  541. --- xlock.new/xlock.c    Fri May 18 19:01:46 1990
  542. ***************
  543. *** 29,34 ****
  544. --- 29,40 ----
  545.    *               Mountain View, CA  94043
  546.    *
  547.    * Revision History:
  548. +  * 18-May-90: Raise window on reinit() - takes care of dialog boxes that pop
  549. +  *              up on top of xlock window.
  550. +  * 09-May-90: xlock will only install a colormap if the -install option is used
  551. +  *              The default action is to attempt allocate all colorcells out
  552. +  *              of the default colormap. This will satisfy ICCCM compliant
  553. +  *              window managers.
  554.    * 18-Feb-90: added SunOS3.5 fix.
  555.    *          changed -mono -> -color, and -saver -> -lock.
  556.    *          allow non-locking screensavers to display on remote machine.
  557. ***************
  558. *** 148,153 ****
  559. --- 154,162 ----
  560.   void        (*callback) () = NULL;
  561.   void        (*init) () = NULL;
  562.   
  563. + unsigned long Pixels[256];    /* array of colormap indices for HSB colors */
  564. + int           NPixels;
  565. + 
  566.   char       *pname;        /* argv[0] */
  567.   Display       *dsp = NULL;        /* server display connection */
  568.   int        screen;        /* current screen */
  569. ***************
  570. *** 170,175 ****
  571. --- 179,185 ----
  572.   char       *fontname = NULL;    /* the font used in the password screen */
  573.   int        lockprocindex;    /* which proc in LockProcs is selected */
  574.   Bool        color = -1;        /* color or mono */
  575. + Bool        install = -1;    /* install colormap, or use default */
  576.   Bool        lock = -1;        /* locked or just saver mode */
  577.   Bool        allowroot = -1;    /* allow root password to unlock? */
  578.   Bool        xhost = -1;        /* whether or not to lock out all hosts */
  579. ***************
  580. *** 193,198 ****
  581. --- 203,209 ----
  582.   #define DEFAULT_FONTNAME    "-*-lucida-medium-r-normal-sans-24-*-*-*-*-*-*-*"
  583.   #define BACKUP_FONTNAME        "fixed"
  584.   #define DEFAULT_COLOR        True
  585. + #define DEFAULT_INSTALL        False
  586.   #define DEFAULT_LOCK        True
  587.   #define DEFAULT_ALLOWROOT    False
  588.   #define DEFAULT_XHOST        True
  589. ***************
  590. *** 343,349 ****
  591.       if (screensaver)
  592.       XSetScreenSaver(dsp, timeout, interval, blanking, exposures);
  593.       XDestroyWindow(dsp, w);
  594. !     if (color)
  595.       XUninstallColormap(dsp, cmap);
  596.       XFlush(dsp);
  597.       XCloseDisplay(dsp);
  598. --- 354,360 ----
  599.       if (screensaver)
  600.       XSetScreenSaver(dsp, timeout, interval, blanking, exposures);
  601.       XDestroyWindow(dsp, w);
  602. !     if (color && install)
  603.       XUninstallColormap(dsp, cmap);
  604.       XFlush(dsp);
  605.       XCloseDisplay(dsp);
  606. ***************
  607. *** 547,554 ****
  608.       do {
  609.       while (!XPending(dsp)) {
  610.           callback();
  611. !         if (reinit())
  612.           init(dsp, w, gc, color, inittime, count);
  613.       }
  614.       XNextEvent(dsp, &event);
  615.       } while (event.type != ButtonPress && event.type != KeyPress);
  616. --- 558,567 ----
  617.       do {
  618.       while (!XPending(dsp)) {
  619.           callback();
  620. !         if (reinit()) {
  621.           init(dsp, w, gc, color, inittime, count);
  622. +             XRaiseWindow(dsp, w);
  623. +         }
  624.       }
  625.       XNextEvent(dsp, &event);
  626.       } while (event.type != ButtonPress && event.type != KeyPress);
  627. ***************
  628. *** 573,581 ****
  629.       for (i = 1; i < NUMPROCS; i++)
  630.       fprintf(stderr, " | %s", LockProcs[i].cmdline_arg);
  631.       fprintf(stderr, "]\n");
  632. !     fprintf(stderr, "\t%s\n\t%s\n",
  633.           "[-time n] [-count n] [-nice n] [-font f]",
  634. !     "[-color] [-lock] [-root] [-xhost] [-screensaver] [-echokeys] [-v]");
  635.       exit(1);
  636.   }
  637.   
  638. --- 586,595 ----
  639.       for (i = 1; i < NUMPROCS; i++)
  640.       fprintf(stderr, " | %s", LockProcs[i].cmdline_arg);
  641.       fprintf(stderr, "]\n");
  642. !     fprintf(stderr, "\t%s\n\t%s\n\t%s\n",
  643.           "[-time n] [-count n] [-nice n] [-font f]",
  644. !         "[-color] [-install] [-lock] [-root] [-xhost] [-screensaver]",
  645. !         "[-echokeys] [-v]");
  646.       exit(1);
  647.   }
  648.   
  649. ***************
  650. *** 593,598 ****
  651. --- 607,614 ----
  652.       }
  653.       }
  654.       usage("Unknown Mode.");
  655. + 
  656. +     /* NOTREACHED */
  657.   }
  658.   
  659.   
  660. ***************
  661. *** 648,653 ****
  662. --- 664,671 ----
  663.           display = argv[i];
  664.       } else if (!strncmp("-color", s, n)) {
  665.           color = !DEFAULT_COLOR;
  666. +     } else if (!strncmp("-install", s, n)) {
  667. +         install = !DEFAULT_INSTALL;
  668.       } else if (!strncmp("-lock", s, n)) {
  669.           lock = !DEFAULT_LOCK;
  670.       } else if (!strncmp("-root", s, n)) {
  671. ***************
  672. *** 760,775 ****
  673. --- 778,805 ----
  674.       if (font == NULL)
  675.           error("%s: can't even find %s!!!\n", BACKUP_FONTNAME);
  676.       }
  677. +     if (install == -1)
  678. +     install = XGetBoolDefault("install", DEFAULT_INSTALL);
  679.       screen = DefaultScreen(dsp);
  680.       if (color)
  681.       color = (DisplayCells(dsp, screen) > 2);
  682.       root = RootWindow(dsp, screen);
  683.   
  684. +     if (color) {
  685. +     if (XCreateHSBColormap(dsp, screen, &cmap, install,
  686. +         0.0, 1.0, 1.0, 1.0, 1.0, 1.0, True,
  687. +         Pixels, &NPixels) != Success)
  688. +         error("%s: couldn't match PseudoColor visual.");
  689. +     } else
  690. +     cmap = DefaultColormap(dsp, screen);
  691. + 
  692.       if (verbose) {
  693.       fprintf(stderr, "display = %s\n", display);
  694.       fprintf(stderr, "font = %s\n", fontname);
  695.       fprintf(stderr, "mode = %s\n", LockProcs[lockprocindex].cmdline_arg);
  696.       fprintf(stderr, "color = %s\n", sbool(color));
  697. +     fprintf(stderr, "#colors = %d\n", NPixels);
  698. +     fprintf(stderr, "install = %s\n", sbool(install));
  699.       fprintf(stderr, "lock = %s\n", sbool(lock));
  700.       fprintf(stderr, "root = %s\n", sbool(allowroot));
  701.       fprintf(stderr, "xhost = %s\n", sbool(xhost));
  702. ***************
  703. *** 779,790 ****
  704.       fprintf(stderr, "count = %d\n", count);
  705.       fprintf(stderr, "nice = %d\n", nicelevel);
  706.       }
  707. -     if (color) {
  708. -     if (XCreateHSBColormap(dsp, screen, &cmap, DisplayCells(dsp, screen),
  709. -                    0.0, 1.0, 1.0, 1.0, 1.0, 1.0, True) != Success)
  710. -         error("%s: couldn't create colormap.");
  711. -     } else
  712. -     cmap = DefaultColormap(dsp, screen);
  713.   
  714.       black.pixel = BlackPixel(dsp, screen);
  715.       XQueryColor(dsp, cmap, &black);
  716. --- 809,814 ----
  717. ***************
  718. *** 803,815 ****
  719. --- 827,849 ----
  720.       xswa.background_pixel = black.pixel;
  721.       xswa.event_mask = KeyPressMask | ButtonPressMask;
  722.   
  723. + #ifdef SABER
  724.       w = XCreateWindow(dsp, root,
  725.                 0, 0,
  726. +               500,
  727. +               500,
  728. +               0, CopyFromParent, InputOutput, CopyFromParent,
  729. +               CWCursor | CWOverrideRedirect |
  730. +               CWBackPixel | CWEventMask, &xswa);
  731. + #else
  732. +     w = XCreateWindow(dsp, root,
  733. +               0, 0,
  734.                 DisplayWidth(dsp, screen),
  735.                 DisplayHeight(dsp, screen),
  736.                 0, CopyFromParent, InputOutput, CopyFromParent,
  737.                 CWCursor | CWOverrideRedirect |
  738.                 CWBackPixel | CWEventMask, &xswa);
  739. + #endif
  740.   
  741.       xswa.cursor = XCreateFontCursor(dsp, XC_target);
  742.       xswa.background_pixel = white.pixel;
  743. ***************
  744. *** 837,843 ****
  745. --- 871,879 ----
  746.       XGetScreenSaver(dsp, &timeout, &interval, &blanking, &exposures);
  747.       XSetScreenSaver(dsp, 0, 0, 0, 0);    /* disable screen saver */
  748.       }
  749. + #ifndef SABER
  750.       GrabKeyboardAndMouse();
  751. + #endif
  752.   
  753.       if (lock) {
  754.       if (xhost)
  755. ***************
  756. *** 847,853 ****
  757.       nice(nicelevel);
  758.       srandom(getpid());
  759.   
  760. !     if (color)
  761.       XInstallColormap(dsp, cmap);
  762.   
  763.       if (lock)
  764. --- 883,889 ----
  765.       nice(nicelevel);
  766.       srandom(getpid());
  767.   
  768. !     if (color && install)
  769.       XInstallColormap(dsp, cmap);
  770.   
  771.       if (lock)
  772. diff -c xlock.orig/xlock.man xlock.new/xlock.man
  773. *** xlock.orig/xlock.man    Thu Jun  7 09:49:42 1990
  774. --- xlock.new/xlock.man    Wed Jun  6 16:45:38 1990
  775. ***************
  776. *** 44,49 ****
  777. --- 44,52 ----
  778.   [
  779.   .B \-v
  780.   ]
  781. + [
  782. + .B \-install
  783. + ]
  784.   
  785.   .SH DESCRIPTION
  786.   .B xlock
  787. ***************
  788. *** 179,184 ****
  789. --- 182,194 ----
  790.   .TP 5
  791.   .B \-v
  792.   Verbose mode, tells what options it is going to use.
  793. + .TP 5
  794. + .B \-install
  795. + Forcibly install the colormap.
  796. + This violates ICCCM standards and will not work with ICCCM
  797. + compliant window managers.
  798. + This option exists for backward compatibility with older
  799. + versions of xlock.
  800.   
  801.   .SH BUGS
  802.   "kill -KILL
  803. EOF of xlock.patch
  804. echo "     691    2686   20177 xlock.patch"
  805. wc xlock.patch
  806.  
  807.  
  808. dan
  809. ----------------------------------------------------
  810. O'Reilly && Associates   argv@sun.com / argv@ora.com
  811. Opinions expressed reflect those of the author only.
  812.